Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Completed Homework 0 on MOR100C-21 #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

liamboone
Copy link

I wasn't quite sure what was meant by "check in," but I've committed to my fork and here's a pull request. Quite new to git/github

Liam

@betajippity
Copy link
Member

Pull request acknowledged!

(My bad, I said earlier that we didn't want pull requests to the master, but we actually do.)

On Tuesday, September 18, 2012 at 9:11 PM, liamboone wrote:

I wasn't quite sure what was meant by "check in," but I've committed to my fork and here's a pull request. Quite new to git/github
Liam
You can merge this Pull Request by running:
git pull https://github.com/liamboone/Homework0 master
Or view, comment on, or merge it at:
https://github.com/CIS565-Fall-2012/Homework0/pull/1
Commit Summary
Added Screenshot as directed, also had to use the glut32.lib fix sugg…
ACTUALLY added screenshot...

File Changes
M HW0_WIN/CUDATEST/main.cpp (5)
A HW0_WIN/Debug/CUDATEST.exe (0)
A HW0_WIN/Debug/CUDATEST.ilk (0)
A HW0_WIN/Debug/CUDATEST.pdb (0)
A HW0_WIN/Debug/freeglut.dll (0)
A HW0_WIN/Debug/glut32.dll (0)
A HW0_WIN/shared/freeglut/glut32.dll (0)
A HW0_WIN/shared/freeglut/lib/glut32.lib (0)
A proof.png (0)

Patch Links
https://github.com/CIS565-Fall-2012/Homework0/pull/1.patch
https://github.com/CIS565-Fall-2012/Homework0/pull/1.diff


Reply to this email directly or view it on GitHub (https://github.com/CIS565-Fall-2012/Homework0/pull/1).

@betajippity betajippity reopened this Sep 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants